Skip to content
This repository has been archived by the owner on Sep 7, 2018. It is now read-only.

Summary headline of report in green or red. #149

Closed
wants to merge 1 commit into from
Closed

Summary headline of report in green or red. #149

wants to merge 1 commit into from

Conversation

petermichaux
Copy link

If all tests that are shown above the fold in the report page are green then there is no red immediately visible to smack me in the face that there was an error hiding below the fold. I actually have to read the words "Test failures!" which is hard work.

@cjohansen
Copy link
Member

Thanks, Peter! I'm a little confused though, we already have this in: busterjs/buster-test@980b6e7 There's also another change that moves the results to the top. I'm guessing you're working from 0.4.5 from busterjs.org?

@petermichaux
Copy link
Author

Yes I was using a file from busterjs.org that did not have the .stats.success and .stats.failure that appear in the changeset you linked to.

I'll try the "Close & comment" button below and see what happens.

@cjohansen
Copy link
Member

Btw, I just uploaded 0.4.6 to busterjs.org: http://busterjs.org/download/

@petermichaux
Copy link
Author

The 0.4.6 and 0.4.5 version of buster-test.css on the download page are identical.

@cjohansen
Copy link
Member

Sorry, fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants